Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request 1478 #1737

Merged
merged 7 commits into from
Jan 30, 2015
Merged

Feature request 1478 #1737

merged 7 commits into from
Jan 30, 2015

Conversation

HugzM
Copy link

@HugzM HugzM commented Jan 7, 2015

I have worked on request #1478 about underline position.

Now the underline position is based on TabstripPlacement property of tabControl.

Example:
if TabstripPlacement is set to Left -> Underline is located at right
Right -> underline is located at left.

About the migration of IsUnderlined Property to dependency property in control it's now not possible because the attached property can be applied on "basic" wpf Tabcontrol.

I can bring the IsUnderlined Property in MetroTabControlComponent but in this case the basic TabControl will not have the ability to have underlined items and i guess it's not the expected behaviors.

Note: Sorry I'm new contributor on github and I havent create a branch to fix my precedent item #1431 that the reason why the commits appears on this pull request . I'll be more careful in the future

@punker76
Copy link
Member

punker76 commented Jan 9, 2015

@HugzM can you separate your changes in different pr's? thx (this pr includes the changes from #1431)

@HugzM
Copy link
Author

HugzM commented Jan 12, 2015

@punker . Ok I cancelled my wrong commit about #1431 on master . This branch is now cleaned

@punker76
Copy link
Member

@HugzM ok, but can you please squash your commits and make the commit message more descriptive?

@punker76 punker76 added this to the 1.1.0 milestone Jan 30, 2015
punker76 added a commit that referenced this pull request Jan 30, 2015
@punker76 punker76 merged commit 2277a1b into MahApps:master Jan 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants